Thoughts?

@dordsor21 by offline player i meant player instance that returns false for isOnline(), not necessarily OfflinePlayer instance.
See #3183

@dordsor21 by offline player i meant player instance that returns false for isOnline(), not necessarily OfflinePlayer instance. See #3183

Altered

I don't think returning null here is a good idea, especially if function isn't marked with nullable itself. I think it's better to return noop attachment or proper attachment but not add it to the hashmap.

I don't think returning null here is a good idea, especially if function isn't marked with nullable itself. I think it's better to return noop attachment or proper attachment but not add it to the hashmap.

The attachment is only used for setting permissions (e.g. when toggling bypass) so null is acceptable here. I've marked the method as nullable as well

Could this PR be merged now?